Skip to content
This repository has been archived by the owner on Dec 22, 2022. It is now read-only.

Use std::size_t in graph loader #132

Merged
merged 8 commits into from
Apr 18, 2022
Merged

Use std::size_t in graph loader #132

merged 8 commits into from
Apr 18, 2022

Conversation

maawad
Copy link
Member

@maawad maawad commented Apr 17, 2022

  • Fix a bug when NDEBUG is defined, causing calls inside asserts to get discarded
  • Assume std::size_t when loading market file
  • Throw exceptions for overflow/underflow cases

@maawad maawad requested a review from neoblizz April 17, 2022 19:07
@neoblizz neoblizz merged commit c9626f0 into gunrock:dev Apr 18, 2022
@maawad maawad deleted the bug-market-loader branch April 18, 2022 06:57
@neoblizz neoblizz added the 🐛 bug Something isn't working label Jun 26, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
🐛 bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants